Skip to content

Conversation

lorenzwalthert
Copy link
Collaborator

@lorenzwalthert lorenzwalthert commented Oct 25, 2020

Remove the majority of package prefixes to see if it brings speed gain.

5dc4a45

  • cache_appyling: 0.106477 -> 0.10819 (2%)
  • cache_recording: 1.58938 -> 1.65327 (4%)
  • without_cache: 4.31798 -> 4.32099 (0%)

e59073a

  • cache_appyling: 0.10285 -> 0.0910904 (-11%)
  • cache_recording: 1.33579 -> 1.18051 (-12%)
  • without_cache: 3.41733 -> 3.06976 (-10%)

697d4b5

  • cache_appyling: 0.10285 -> 0.101439 (-1%)
  • cache_recording: 1.33579 -> 1.33368 (0%)
  • without_cache: 3.41733 -> 3.52667 (3%)

with https://github.com/lorenzwalthert/touchstone

  • cache_applying: 0.03 -> 0.03 (-12.2%)
  • cache_recording: 1.01 -> 1 (-1.4%)
  • without_cache: 2.97 -> 2.95 (-0.6%)

@codecov-io
Copy link

codecov-io commented Oct 25, 2020

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@lorenzwalthert lorenzwalthert force-pushed the remove-pkg-prefix branch 2 times, most recently from 980b7eb to e59073a Compare November 1, 2020 15:55
@lorenzwalthert
Copy link
Collaborator Author

cc: @krlmlr

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants